Skip to content

Multiselect for instance deletion#3268

Open
huru-ukko wants to merge 5 commits intoflowable:mainfrom
alfameCom:multiselect_for_instance_deletion
Open

Multiselect for instance deletion#3268
huru-ukko wants to merge 5 commits intoflowable:mainfrom
alfameCom:multiselect_for_instance_deletion

Conversation

@huru-ukko
Copy link

@huru-ukko huru-ukko commented Apr 8, 2022 •
edited
Loading

Check List:

  • Unit tests: NO
  • Documentation: NO
  • Added the possibility to multiselect instances under Admin app
  • Added the possibility to delete multiple instances at a time from the instances listing. They are first terminated with a set reason, and then deleted.
  • List of failed deletions , links to failed processes that open in a new window

flowable-process-instances

Tuomas Pesola added 4 commits April 7, 2022 15:27
@huru-ukko huru-ukko marked this pull request as ready for review April 8, 2022 12:19
@huru-ukko
Copy link
Author

huru-ukko commented May 3, 2022

Is there something i need to do in before a maintainer will take a look at this pr?

@tijsrademakers
Copy link
Contributor

tijsrademakers commented May 3, 2022

Hi @wuahi ,

We are currently looking at extending the REST API with some bulk action support including terminating / deleting process instances. When this would be available no loop of calling the REST API per process instance from the Admin web application is needed anymore. So we would first like to explore the bulk action support in the REST API a bit more.

Best regards,

Tijs

@huru-ukko
Copy link
Author

huru-ukko commented May 3, 2022

Thanks for the quick response and this is lovely to hear! Any time estimate for this extension of the REST API?

@filiphr
Copy link
Contributor

filiphr commented Jul 11, 2022

Hey @wuahi, we have added this functionality on main.

The endpoint for deleting process instances is here and the PR that added this functionality is #3295.

Can you please rebase your branch on main and adapt your changes to use the new REST API?

@huru-ukko
Copy link
Author

huru-ukko commented Jul 14, 2022

Hey @wuahi, we have added this functionality on main.

The endpoint for deleting process instances is here and the PR that added this functionality is #3295.

Can you please rebase your branch on main and adapt your changes to use the new REST API?

Nice to hear! Yeah sure. I will try to do it within a week. Thanks for the info :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@janihaglund janihaglund janihaglund approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@huru-ukko @tijsrademakers @filiphr @janihaglund